Skip to content

Add pipeline for linting and formatting#146

Merged
jakubno merged 1 commit intomainfrom
require-lint
Aug 31, 2025
Merged

Add pipeline for linting and formatting#146
jakubno merged 1 commit intomainfrom
require-lint

Conversation

@jakubno
Copy link
Copy Markdown
Member

@jakubno jakubno commented Aug 29, 2025 •
edited
Loading

Setup linting and formatting in all packages

@jakubno jakubno force-pushed the require-lint branch 4 times, most recently from 06b06d8 to f5a7417 Compare August 31, 2025 15:44
@jakubno jakubno changed the title Require lint Add pipeline for linting and formatting Aug 31, 2025
@jakubno jakubno marked this pull request as ready for review August 31, 2025 17:36
cursor[bot]
cursor bot reviewed Aug 31, 2025
View reviewed changes
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Example File Uses Unavailable Template ID

The example.mts file was updated to use a hardcoded template ID 'bwyvo5fk343pbvxst536' and beta functionality. This ID likely belongs to an internal test template, making the example unreliable for general users without access to it. This change also seems unrelated to the commit's stated purpose.

js/example.mts#L1-L35

code-interpreter/js/example.mts

Lines 1 to 35 in 45ce5ca

import { config } from 'dotenv'
import { Sandbox } from './dist'
function log(...args: any[]) {
console.log(...args)
}
config()
const sbx = await Sandbox.create('bwyvo5fk343pbvxst536')
log('ℹ️ sandbox created', sbx.sandboxId)
await sbx.runCode('x = 1')
log('Sandbox code executed')
const sandboxId = await sbx.betaPause()
log('Sandbox paused', sandboxId)
// Resume the sandbox from the same state
const sameSbx = await Sandbox.connect(sbx.sandboxId)
log('Sandbox resumed', sameSbx.sandboxId)
const execution = await sameSbx.runCode('x+=1; x')
// Output result
log(execution.text)
log(execution.error)
if (execution.text !== '2') {
log('Test failed:', 'Failed to resume sandbox')
throw new Error('Failed to resume sandbox')
}
log('Sandbox resumed successfully')
await sbx.kill()
log('Sandbox deleted')

Fix in Cursor Fix in Web

Comment @cursor review or bugbot run to trigger another review on this PR

@jakubno jakubno merged commit 09efac2 into main Aug 31, 2025
9 checks passed
@jakubno jakubno deleted the require-lint branch August 31, 2025 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursor cursor[bot] cursor[bot] left review comments

@mlejva mlejva mlejva approved these changes

@ValentaTomas ValentaTomas Awaiting requested review from ValentaTomas ValentaTomas is a code owner

@0div 0div Awaiting requested review from 0div

@mishushakov mishushakov Awaiting requested review from mishushakov mishushakov is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jakubno @mlejva